-
Notifications
You must be signed in to change notification settings - Fork 14
Add xinterpdf #18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add xinterpdf #18
Conversation
pavoljuhas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good start, but needs some updates.
products/xinterpdf.rst
Outdated
| xINTERPDF | ||
| ########## | ||
|
|
||
| xINTERPDF is a Python GUI program for analyzing X-ray pair distribution function (PDF) data of organic compounds collected at synchrotron and/or laboratory X-ray sources. It uses DiffPy-CMI as a backend for simulation of PDFs. Currently it supports (1) The study of intermolecular interaction (e.g. hydrogen bonds) by subtracting out the scattering signal(s) of single molecule(s) in real space. (2) The PDF model fit of the crystalline organic compound using the method proposed by Prill et al. (*J. Appl. Cryst.*, **48**, 171-178, 2015.) (3) The phase quantification of physical mixtures of organics. (4) Generate Score/Scree plots based on Principle Component Analysis (PCA). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- consider changing to numbered list
- convert J. Appl. Cryst., ... to a hyperlink to http://dx.doi.org/10.1107/S1600576714026454
products/xinterpdf.rst
Outdated
|
|
||
| User Guide | ||
| ------------- | ||
| Please see `User manual <https://github.com/curieshicy/xINTERPDF/blob/master/xINTERPDF_User_Guide_20180615.pdf>`__ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please copy the user guide into the static_root/doc/xinterpdf folder and link it as
<../doc/xinterpdf/xINTERPDF_User_Guide_20180615.pdf>`__
It is better to have all documentation served from diffpy.org.
products/xinterpdf.rst
Outdated
| Please see `User manual <https://github.com/curieshicy/xINTERPDF/blob/master/xINTERPDF_User_Guide_20180615.pdf>`__ | ||
| for more details. | ||
|
|
||
| Reference |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be References
products/xinterpdf.rst
Outdated
| papers: | ||
|
|
||
| * Chenyang Shi, “xINTERPDF: a GUI program for analyzing intermolecular pair distribution functions of organic compounds from X-ray total scattering data”, under review, *J. Appl. Cryst.*, 2018. | ||
| * Pavol Juhás, Christopher L. Farrow, Xiaohao Yang, Kevin R. Knox, and Simon J. L. Billinge, “Complex modeling: a strategy and software program for combining multiple information sources to solve ill posed structure and nanostructure inverse problems”, *Acta Crystallogr. A*, **71**, 562-568, 2015. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please replace with abbreviation |citeJuhasAca15|. You will also need to add the following line to the end of the rst file:
.. include:: ../abbreviations.txt
products/xinterpdf.rst
Outdated
| we ask you to acknowledge the use of xINTERPDF by citing the following | ||
| papers: | ||
|
|
||
| * Chenyang Shi, “xINTERPDF: a GUI program for analyzing intermolecular pair distribution functions of organic compounds from X-ray total scattering data”, under review, *J. Appl. Cryst.*, 2018. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add abbreviation |citeShiJac18| into the abbreviations.txt file using the same style as other entries there.
Then cite here as |citeShiJac18|. This will let us reuse it in the Publications page when accepted.
GNUmakefile
Outdated
| @@ -0,0 +1,2 @@ | |||
| include Makefile | |||
| PUBLISHBRANCH = gh-pages | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete this file. It is intended for your working copy only and should not be committed.
Its purpose is to setup preview page under your account rather than to diffpy.org.
|
Pavol, |
And fix trailing blanks from `git diff --check`. No change in content.
Add link to diffpy-users. Fix few typos.
|
@curieshicy - I made a few corrections, see 541c76b...172b0c8. |
Pavol,
I followed your instructions and arrived at this pull request step. If you see issues, please free to reject.